[log] Add debug logging to server/handlers.go#411
Merged
Conversation
- Add logHandlers debug logger with namespace server:handlers - Add logging for OAuth discovery handler lifecycle - Add logging for close endpoint handler lifecycle - Add logging for shutdown validation and execution - 7 new debug log statements for troubleshooting Follows project logging conventions from AGENTS.md
3 tasks
9 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Added debug logging to
internal/server/handlers.goto improve troubleshooting and development visibility.Changes
Logger Declaration
logHandlersdebug logger with namespaceserver:handlersfollowing the project'spkg:filenameconventionLogging Enhancements (7 new statements)
OAuth Discovery Handler
Close Endpoint Handler
Benefits
DEBUG=server:handlersis setTesting
The changes follow the established logging patterns in the codebase and maintain consistency with other handlers.
Compliance
✅ Exactly 1 file modified (focused, single-file PR)
✅ No test files modified
✅ Logger naming follows
pkg:filenameconvention✅ Logger arguments have no side effects
✅ Logging messages are meaningful and helpful
✅ No duplicate logging with existing logs
✅ Follows AGENTS.md guidelines